Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update RGL to v15 with simple velocity distortion and new LiDAR HesaiAT128 #202

Merged
merged 3 commits into from
Oct 3, 2023

Conversation

msz-rai
Copy link
Contributor

@msz-rai msz-rai commented Sep 23, 2023

PR contains:

msz-rai and others added 2 commits September 23, 2023 20:25
* Adjust Plugin to new API call for setting range

* Add pattern to Hesai AT128

* Add minRange property

* Enable setting center of measurement offset in all of the directions

* Update prefabs

* Update docs

* Add prefab & correct pattern

* Add binary for Linux

* Review changes

* Fix RGL binary, revert changes in LidarSceneDevelop

* Add HesaiAT128 to LidarSceneDevelop scene
* Add time offsets feature

* fix copypaste error

* move lasers offset from library to calculating

* add velocity distortion first approach

* add rotation to distortion

* small fixes

* back to laserArrayOffsets data

* add data for velodyne 16 and 32

* add PandarQT data

* add Pandar40 data

* add placeholders for velodyne 128 and ouster

* add velodyne vls-128 configuration

* PR fixes v1

* fix minor bug

* catch bug with units

* update lidars prefabs

* movetransform assigment on tick

* fix bug with radians

* move transforming do distord method

* Delete usless member

* Update RGL binary after rebase

* Fix RGL velocity distortion v1 (#188)

* Adjust plugin to fixed native RGL version

* Calculate velocities in sensor-local coordinate frame

* Fix offsets calculation

* Add time offsets to Hesai AT128

* Update VLS128 time offsets

* Update lidar prefabs

* Update RGL binary

* Disable velocity distortion by default

* Add sample scene for tests

---------

Co-authored-by: Mateusz Szczygielski <[email protected]>
Co-authored-by: Mateusz Szczygielski <[email protected]>
Copy link
Collaborator

@mackierx111 mackierx111 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great! LGTM

@mackierx111 mackierx111 merged commit f7fc2f3 into main Oct 3, 2023
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants